popover: Stop using gtk_widget_input_shape_combine_region()

We control the surface anyway, we can as well set the input shape on it
directly.

widget: Remove set_csd_input_shape

Replace the only caller (unsurprisingly from gtkwindow.c) with a direct
call to gdk_surface_input_shape_combine_region.

widget: Remove surface member

Only GtkNatives are supposed to have a surface, so use the widget's
native.

range: Remove an outdated comment

What even is a CList.

range: Inline function into only caller

range: Clean up compute_slider_position()

menusectionbox: Avoid a gtk_widget_destroy() call

pathbar: Inherit from GtkWidget

colorchooser: Fix typo in checkerboard color

inspector: Remove double borders in recorder

headerbar: Replace gtk_widget_destroy() call

headerbar: Remove size request on icon button

Doesn't make sense that we force this one button to be this big.

shortcutlabel: Remove unnecessary includes

stackswitcher: Avoid calling gtk_widget_destroy()

modelbutton: Fix wrong function name in doc comment

placessidebar: Convert popover menu to proper GtkPopoverMenu

placessidebar: Avoid calling gtk_widget_destroy()

pathbar: Avoid calling gtk_widget_destroy()

headerbar: Avoid calling gtk_widget_destroy()

popovermenubar: Avoid calling gtk_widget_destroy()

emojichooser: Avoid using gtk_widget_destroy()

popovermenu: Avoid a gtk_widget_destroy() call

colorchooserwidget: Get rid of a gtk_widget_destroy() call

testsuite: Remove outdated exclusions

There were mentions of GtkCList and GtkColorSelection
in these test. The memories!

Merge branch 'spinning-spinner' into 'master'

Rename GtkSpinner::active to ::spinning

See merge request GNOME/gtk!1475

Rename GtkSpinner::active to ::spinning

And add a setter and getter. The old name was
confusing with the widget state of the same
name. 'Active' is just too overloaded.

Merge branch 'kill-rtl-media-icons' into 'master'

Drop rtl variants of media-playback-start

See merge request GNOME/gtk!1474

Merge branch 'kill-popover-relative-to' into 'master'

Drop GtkPopover::relative-to

See merge request GNOME/gtk!1471

Drop rtl variants of media-playback-start

According to #2469, they are unwanted.

popover: Drop ::relative-to

It is enough to just set the parent (and make the parent
call gtk_native_check_resize in size_allocate).

This commit removes the relative_to argument to the
constructors of GtkPopover and GtkPopoverMenu, and
updates all callers.

Merge branch 'surface-get-mapped' into 'master'

Add gdk_surface_get_mapped

See merge request GNOME/gtk!1473

Drop gdk_surface_is_visible

It has been replaced by gdk_surface_get_mapped.

Replace gdk_surface_is_visible by _get_mapped

The property  is called ::mapped, and we want to get
to standard getter naming.

Add gdk_surface_get_mapped

Returns the value of the ::mapped property. This is
a direct replacement for gtk_surface_is_visible.

Merge branch 'getters-and-setters' into 'master'

gdk:Getters and setters

See merge request GNOME/gtk!1472

Add missing GdkDrag getters

surface: Add a getter for autohide

gdk: Drop GdkDevice::input-mode and rename ::input-source

Drop the input-mode, since it only makes sense for
floating devices, which we don't have anymore. And renamt
::input-source to ::source, to match the getter.

Update all users.

device: Add missing getters

This adds getters for a few properties that were
missing them, and renames the input-mode and input-source
properties to match their existing getters.

Add a getter for GdkAppLaunchContext::display

Fix the build with docs

This is fallout from filechooser api changes.

Merge branch 'wip/ebassi/filechooser-new' into 'master'

Clean up the GtkFileChooser API

Closes #2455

See merge request GNOME/gtk!1454

Merge branch '138-gtk4-install-valgrind-suppressions' into 'master'

build: Install Valgrind suppressions files

Closes #138

See merge request GNOME/gtk!1467

Merge branch 'matthiasc/for-master' into 'master'

broadway: Remove SURFACE_IS_TOPLEVEL

See merge request GNOME/gtk!1469

broadway: Remove SURFACE_IS_TOPLEVEL

All surfaces are toplevels now.

build: Install Valgrind suppressions files

This is so that other programs linking to GTK can use GTK's suppressions
files when performing their own Valgrind analysis.

Closes: #138

Update POTFILES.in

Merge branch 'wip/otte/dnd' into 'master'

Get rid of GdkAtom

See merge request GNOME/gtk!1465

gdk: Remove GdkAtom

finally!!!!!

win32: Get rid of GdkAtom

wayland: Replace final mention of GdkAtom

x11: Get rid of GdkAtom and APIs supporting it.

replace all uses with const char * (non-interned).
Also remove a lot fo juggling from atom to GdkAtom to string and back.

The X Atom hash table is now mapping to (again, non-interned) strings.

device: gdk_device_list_axes() => gdk_device_get_axis_names()

Turn a GList of GdkAtom into a char ** - and rename the function to not
cause problems.

gdk: Remove gdk_text_property_to_utf8_list_for_display()

A lot of files became empty now, so they have been removed, which makes
this commit seem larger than it is.

Don't use GdkAtom where const char * is used

Those are all forgotten transitions while updating code to new APIs.

gdk: Remove gdk_utf8_to_string_target()

Only keep the X11 version around in the backend.

selection: Remove GtkSelectionData

droptarget: Remove selectiondata read functions

They are unused now.

filechooserwidget: Stop using selection data

Use gdk_drop_read_async()

x11: Register DND atoms with all other atoms

It's 2020, there's no need to be restrained with registering atoms.

gdk: Remove gdk_surface_register_dnd()

All surfaces are expected to be DND surfaces from creation.

Merge branch 'x11-dnd-fixes' into 'master'

X11 dnd fixes

See merge request GNOME/gtk!1464

Add detail to gdk_drag_begin docs

Mention that GTK keeps a reference while the drag
operation is ongoing.

x11: Fix dnd coordinate handling

We were not properly converting the coordinates we
got to root coordinates. This was showing up as offsets
between the actual drop target and the area where drops
can happen, e.g. when dragging over a stack switcher
to switch pages.

x11: Export gdk_x11_surface_get_root_coords privately

This lets us avoid a roundtrip through the surface vfuncs.

x11: Keep a ref on GdkDrag objects

It is expected that backends keep a ref on the GdkDrag
objects that they create as long as the drag is ongoing.

x11: Avoid crashes in dnd

We were forgetting to clean up the ::xevent signal
handler in some error cases. Move the signal connection
later, when we know the drag is going forward, and
use g_signal_connect_object to make sure the signal
handler is not forgotten.

Merge branch 'wip/wayland-fix-popup-grabs' into 'master'

Fix Wayland popup grabs

See merge request GNOME/gtk!1463

wayland/popup: Emit un-withdrawn event earlier

Otherwise grabbing the seat will appear to have failed.

wayland/popup: Remove unnecessary checks when mapping popup

We only call this when we're not already mapped so don't check that. We
also only call this when we should be mapped so don't check that either.

Merge branch 'matthiasc/for-master' into 'master'

Fix a hard-to-spot typo

See merge request GNOME/gtk!1462

Fix a hard-to-spot typo

The symptom caused by this was scales in
popovers not reacting to dragging.

docs: Add migration notes for GtkFileChooser

Merge branch 'matthiasc/for-master' into 'master'

x11: Fix a crash in event handling

See merge request GNOME/gtk!1461

x11: Fix a crash in event handling

This was overlooked in 79b4510c6d687c.

Remove preview widget from GtkFileChooser

The preview widget harks from a platform before time, when we didn't
have GIO, or a thumbnail specification.

Very few applications use it correctly, if at all; it has an horrid hack
to deal with the ownership of the widget's instance when accessed
through the getter function; it messes up the layout of the widget and
its label is less than useful when it comes to file names longer than a
dozen characters; it's a poor substitute for a proper thumbnail view.

Fix gdk docs build

Remove GtkFileChooserEntry:local-only

We don't use it any more in GtkFileChooserWidget.

Remove GtkPlacesSidebar:local-only

We don't use it any more in GtkFileChooserWidget.

Remove GtkPlacesView:local-only

We don't use it any more from the file chooser widget.

Remove GTK_FILE_CHOOSER_ACTION_CREATE_FOLDER

We don't need a whole separate action, now that the file chooser widget
can create folders; we can create use SELECT_FOLDER and create one.

Remove GtkFileChooser:local-only

Now that the whole API goes through GFile we don't have the weird split
between local-only and non-local-only modes.

Remove GtkFileChooser:show-hidden

This is a user setting and a user action, not something that ought to be
programmatically set via the file selection API.

Remove overwrite confirmation machinery from GtkFileChooser

Overwrite confirmation should not be optional, and it should not loop
into application code to create their own dialog and user response.

Drop GtkFileChooser:extra-widget

We have "choices" as a more rational (and portable) API; additionally,
the ownership semantics of the extra widget property getter are a hack.

Remove filename/URI API from GtkFileChooser

GtkFileChooser's API predates GIO by a few years, so it started off with
filenames and URI as character arrays. After introducing GIO as a
dependency, the API included GFile-based entry points.

It's much more appropriate to use GFile everywhere, as we want to
encourage people to use GIO instead of passing random bytes to low level
POSIX API.

See: #2455

Merge branch 'wip/otte/dnd' into 'master'

stuff

See merge request GNOME/gtk!1460

Remove GdkSeatGrabPrepareFunc from headers

It is not used in public api anymore.

Add a warning when a grab fails

This reveals that the grabs for popover menus
always fail now, causing the menus to not properly
hide.

tests: Guard variables with necessary macros

treeviewcolumn: Inline variable into return_if_fail()

testsuite: Don't use g_return_if_fail()

Use g_assert()

testsuite: Don't use return_if_fail()

Use assert() instead

treeview: Guard code by right compiler macros

scrolledwindow: Inline variable into return_if_fail()

iconview: Inline variables into return_if_fail()

levelbar: Guard function by right compiler macros

filesystemmodel: Guard variable with correct compiler flags

combobox: Inline variables into return_if_fail()

comboboxtext: Inline functions into return_if_fail()